Skip to content

[TASK] Use reusable workflows and actions#641

Merged
eliashaeussler merged 1 commit into
mainfrom
task/workflows
May 23, 2026
Merged

[TASK] Use reusable workflows and actions#641
eliashaeussler merged 1 commit into
mainfrom
task/workflows

Conversation

@eliashaeussler

@eliashaeussler eliashaeussler commented May 21, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Chores
    • Consolidated CI/CD workflows for improved pipeline efficiency and maintainability
    • Removed legacy workflow files and introduced a single unified CI workflow covering unit, E2E, and coverage stages
    • Refactored release pipeline to use reusable actions for asset builds, docs deployment, and release creation
    • Updated required status checks for more granular test validation across PHP versions and environments

Review Change Stack

@eliashaeussler eliashaeussler added the maintenance Code base is being maintained label May 21, 2026
@eliashaeussler eliashaeussler self-assigned this May 21, 2026
@coderabbitai

coderabbitai Bot commented May 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@eliashaeussler, we couldn't start this review because you've used your available PR reviews for now.

Your plan currently allows 2 reviews/hour. Refill in 14 minutes and 28 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more review capacity refills, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: edda136a-dba9-43ad-9f40-75bdbc67d410

📥 Commits

Reviewing files that changed from the base of the PR and between ba31448 and fafef83.

📒 Files selected for processing (5)
  • .github/settings.yml
  • .github/workflows/cgl.yaml
  • .github/workflows/ci.yaml
  • .github/workflows/release.yaml
  • .github/workflows/tests.yaml

Walkthrough

The PR modernizes GitHub Actions by consolidating test workflows into a reusable-driven ci.yaml, removing legacy test workflows, refactoring release.yaml to use reusable actions with distinct assets/docs/release jobs, and updating .github/settings.yml required status check contexts.

Changes

CI/CD Workflow Consolidation and Release Modernization

Layer / File(s) Summary
New CI workflow foundation with prepare and checks jobs
.github/workflows/ci.yaml
Adds CI identity, triggers (push to all branches, PRs to main), contents: read permission, a reusable prepare job that gates downstream jobs, and a checks job which depends on prepare and has security-events: write permission when gated.
CI workflow test execution jobs
.github/workflows/ci.yaml
Adds unit-tests (matrix PHP 8.2–8.5, highest/lowest dependency strategies, OS includes for PHP 8.5), e2e-tests (runs test:e2e), and test-coverage (reusable coverage workflow); all run conditionally after prepare.
Required status checks alignment
.github/settings.yml
Replaces generic required status check context names with explicit Composer/npm/repo check contexts, per-PHP-version “Unit tests (...) / Run tests” contexts, and renames the coverage context to “Test coverage / Collect test coverage”; keeps integration_id values unchanged.
Release workflow refactored to reusable actions
.github/workflows/release.yaml
Adds workflow env vars for PHAR_FILE and REF_NAME; replaces inline PHAR/Docker/docs/release steps with assets (build & sign PHAR, build Docker, export artifact name), docs (download artifact, stage PHAR in docs/public/, deploy pages, cache-warmup), and release (reusable create-release action) jobs.

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main objective of the PR: refactoring GitHub Actions workflows to use reusable workflows and actions, which aligns with the substantial changes across multiple workflow files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coveralls

coveralls commented May 21, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 26333809527

Coverage remained the same at 90.01%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 1982
Covered Lines: 1784
Line Coverage: 90.01%
Coverage Strength: 10.18 hits per line

💛 - Coveralls

@eliashaeussler eliashaeussler force-pushed the task/workflows branch 2 times, most recently from e289d6c to a1a3b2b Compare May 21, 2026 14:09
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
Comment thread .github/workflows/cgl.yaml Fixed
@eliashaeussler eliashaeussler force-pushed the task/workflows branch 2 times, most recently from f0f80c6 to 2c2d4d5 Compare May 21, 2026 16:26
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
@eliashaeussler eliashaeussler force-pushed the task/workflows branch 6 times, most recently from ddba1d5 to 3872d50 Compare May 21, 2026 17:10
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
@eliashaeussler eliashaeussler force-pushed the task/workflows branch 2 times, most recently from 25be0a9 to 34847df Compare May 21, 2026 17:49
Comment thread .github/workflows/ci.yaml Fixed
Comment thread .github/workflows/ci.yaml Fixed
@eliashaeussler eliashaeussler force-pushed the task/workflows branch 2 times, most recently from 08a74c7 to a6a05f5 Compare May 22, 2026 08:54
@eliashaeussler eliashaeussler force-pushed the task/workflows branch 2 times, most recently from bc4c38b to d726d74 Compare May 22, 2026 09:05

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🧹 Nitpick comments (1)
.github/workflows/release.yaml (1)

106-107: ⚡ Quick win

Consider gating release on docs completion.

Right now a release can be created even if docs deploy/cache-warmup fails. If docs are part of release readiness, add docs to needs.

Suggested change
-    needs: [assets]
+    needs: [assets, docs]

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 49e4be0e-f5fd-4f9c-836c-02d35aadc3c5

📥 Commits

Reviewing files that changed from the base of the PR and between e445f57 and ba31448.

📒 Files selected for processing (5)
  • .github/settings.yml
  • .github/workflows/cgl.yaml
  • .github/workflows/ci.yaml
  • .github/workflows/release.yaml
  • .github/workflows/tests.yaml
💤 Files with no reviewable changes (2)
  • .github/workflows/cgl.yaml
  • .github/workflows/tests.yaml

Comment thread .github/workflows/release.yaml
@eliashaeussler eliashaeussler merged commit 6886b70 into main May 23, 2026
26 checks passed
@eliashaeussler eliashaeussler deleted the task/workflows branch May 23, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Code base is being maintained

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants